Do an explicit check for bootloader path existence prior

to the one for executable permissions. Also print out the path being
tested.

Signed-off-by: Daniel P, Berrange <berrange@redhat.com>

diff --git a/tools/python/xen/xm/create.py b/tools/python/xen/xm/create.py
index c081560f8e53b05af4b4b05f5e642eb48a345da9..aa0c03cac39678e66dd2872027ce3a3620097e69 100644 (file)
--- a/tools/python/xen/xm/create.py
+++ b/tools/python/xen/xm/create.py
@@ -704,8 +704,10 @@ def configure_hvm(config_image, vals):
     config_image.append(['vncpasswd', vals.vncpasswd])
 
 def run_bootloader(vals, config_image):
+    if not os.access(vals.bootloader, os.F_OK):
+        err("Bootloader '%s' does not exist" % vals.bootloader)
     if not os.access(vals.bootloader, os.X_OK):
-        err("Bootloader isn't executable")
+        err("Bootloader '%s' isn't executable" % vals.bootloader)
     if len(vals.disk) < 1:
         err("No disks configured and boot loader requested")
     (uname, dev, mode, backend) = vals.disk[0]